Skip to content

chore(deps): bump golang.org/x/crypto to v0.52.0#605

Merged
osm6495 merged 2 commits into
mainfrom
devin/1783987982-bump-x-crypto
Jul 14, 2026
Merged

chore(deps): bump golang.org/x/crypto to v0.52.0#605
osm6495 merged 2 commits into
mainfrom
devin/1783987982-bump-x-crypto

Conversation

@kparkinson-ld

@kparkinson-ld kparkinson-ld commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Remediates the open CRITICAL Dependabot alert for golang.org/x/crypto by bumping it from v0.45.0 to v0.52.0 (patched version).

golang.org/x/crypto v0.52.0 requires Go >= 1.25, so this also bumps the module go directive 1.24.0 → 1.25.0 (and drops the now-redundant toolchain go1.24.1 line). CI resolves its Go version from go.mod (go-version-file: go.mod); there are no Dockerfile pinned Go versions to update.

golang.org/x/crypto v0.45.0 -> v0.52.0  (fixes CRITICAL alert)
go 1.24.0 -> 1.25.0 (required by x/crypto v0.52.0)

Transitive golang.org/x/{net,sys,text,term} were pulled forward by go mod tidy as a consequence. The vendor/ tree was regenerated via go mod vendor — hence the large (mechanical) vendored diff.

CI lint compatibility (ci: commit)

The Go 1.25 bump broke the CI go-tests job: the pinned golangci-lint v1.64.4 (built with Go 1.24) refuses to target a go 1.25.0 module. Upgraded the lint step to golangci-lint-action@v8.0.0 + golangci-lint v2.5.0 (built with Go 1.25) and migrated .golangci.yml to the v2 schema (via golangci-lint migrate).

To preserve the prior effective linter behavior (and avoid reformatting unrelated source), the v2-only linters that would otherwise fire on existing code are disabled (funcorder, godoclint, noinlineerr, wsl_v5), and staticcheck is configured to exclude the ST1*/QF1* check groups (v1 had stylecheck disabled). The old goimports local-prefixes setting was dropped because the v1 config used the misspelled key linter-settings, so it was silently ignored and never enforced.

Verified locally: go build ./..., go vet ./..., go test ./..., and golangci-lint run ./... (v2.5.0) all pass. CI green.

Link to Devin session: https://app.devin.ai/sessions/77c5bbaf5b114aac80ad2c021c2b14f4
Requested by: @kparkinson-ld

@kparkinson-ld kparkinson-ld self-assigned this Jul 14, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review July 14, 2026 00:31
@devin-ai-integration
devin-ai-integration Bot requested review from a team July 14, 2026 00:33
@osm6495
osm6495 merged commit 8e56bb1 into main Jul 14, 2026
6 checks passed
@osm6495
osm6495 deleted the devin/1783987982-bump-x-crypto branch July 14, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants